-
Notifications
You must be signed in to change notification settings - Fork 180
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added autoplay next related video feature #428
base: main
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
5aa2679
to
c454974
Compare
@PraveenRaj1833 Exciting to see this PR come through, thank you! It’s on my list to review |
Thank you @shakeelmohamed, It was quite a good learning for me. Waiting for your suggestions on this. |
Hi @shakeelmohamed, did you get a chance to review the code? |
@PraveenRaj1833, I’m just getting to this now. Can you provide a quick outline of the different user flows through this feature? Related to the user flows, are you interested in writing automated tests for this feature? |
hi @PraveenRaj1833, are you interested in writing some tests for this awesome feature? |
Motivation and Context
discussions.
Types of changes
What types of changes does your code introduce? Check all the boxes that apply:
Description
Final checklist:
Go over all the following points and check all the boxes that apply
If you're unsure about any of these, don't hesitate to ask. We're here to help!